Skip to content

Add egui example#164

Merged
wusyong merged 6 commits into
tauri-apps:devfrom
zeerooth:egui-example
May 29, 2023
Merged

Add egui example#164
wusyong merged 6 commits into
tauri-apps:devfrom
zeerooth:egui-example

Conversation

@zeerooth
Copy link
Copy Markdown
Contributor

@zeerooth zeerooth commented May 25, 2023

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

Checklist

  • This PR will resolve #___
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary
  • It can be built on all targets and pass CI/CD.

Other information

This is partially a continuation of BrainiumLLC/cargo-mobile#48

This PR introduces a new template based on an egui example found here https://github.com/rust-mobile/rust-android-examples/tree/main/agdk-egui

It works on Android and Linux desktop, I couldn't test it anywhere else

wusyong
wusyong previously approved these changes May 26, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@wusyong wusyong left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Could you sign the sommit so I could merge the PR.

@zeerooth
Copy link
Copy Markdown
Contributor Author

zeerooth commented May 26, 2023

Thanks! Could you sign the sommit so I could merge the PR.

No problem, I signed the last commit, is it enough?

@wusyong
Copy link
Copy Markdown
Member

wusyong commented May 27, 2023

You'll have to sign all commits I believe.

@zeerooth
Copy link
Copy Markdown
Contributor Author

zeerooth commented May 27, 2023

You'll have to sign all commits I believe.

Alright, I rewrote history and now it seems like every commit is properly verified. Sorry for all of that

@wusyong wusyong merged commit 94560bf into tauri-apps:dev May 29, 2023
@QuantumEntangledAndy
Copy link
Copy Markdown

QuantumEntangledAndy commented Jun 1, 2023

Just trying your equi example with latest install. Had a few issues, on mac and ios.

First I got this error when doing cargo run on the mac

error[E0603]: function `main` is private
   --> gen/bin/desktop.rs:3:18
    |
3   |     mobile_equi::main();
    |                  ^^^^ private function
    |
note: the function `main` is defined here
   --> /Users/awk21/Projects/Coding/Rust/mobile_equi/src/lib.rs:160:1
    |
160 | fn main() {
    | ^^^^^^^^^

For more information about this error, try `rustc --explain E0603`.
error: could not compile `mobile-equi` due to previous error

Which was solvable by adding pub to main

What was more troubling was trying to solve the following on the ios

Undefined symbols for architecture arm64:
  "_start_app", referenced from:
      _main in main.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
note: Building targets in dependency order
note: Run script build phase 'Build Rust Code' will be run during every build because the option to run the script phase "Based on dependency analysis" is unchecked. (in target 'mobile-equi_iOS' from project 'mobile-equi')
** BUILD FAILED **

error: Failed to build via `xcodebuild`
    command ["xcodebuild"] exited with code 65

Not sure where _start_app is or how to add that it. Any ideas?

@QuantumEntangledAndy QuantumEntangledAndy mentioned this pull request Jun 1, 2023
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wusyong wusyong wusyong approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zeerooth @wusyong @QuantumEntangledAndy